Skip to content

Fix msg.value balance accounting in invariant tests - #590

Open
DicksonWu654 wants to merge 1 commit into
a16z:mainfrom
DicksonWu654:fix/top-level-msg-value-balance
Open

Fix msg.value balance accounting in invariant tests#590
DicksonWu654 wants to merge 1 commit into
a16z:mainfrom
DicksonWu654:fix/top-level-msg-value-balance

Conversation

@DicksonWu654

Copy link
Copy Markdown

Summary

  • generate symbolic msg.value only for payable invariant targets
  • apply funded top-level CALL value transfers to sender and recipient balances
  • preserve self-transfer semantics and discard solver-proven underfunded states
  • restore persistent state and transaction-start transient storage after any EVM failure
  • add unit and invariant regressions for transfer, rollback, and positive-value reachability

Validation

  • git diff --check
  • independent static review with gpt-5.6-sol via the non-cyber codex-lb route

Tests were not run because this review was constrained to passive/static inspection and prohibited executing repository code, builds, package managers, or hooks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant